Harshavarma: fix - Job Posting Analytics Graph Issues#5013
Harshavarma: fix - Job Posting Analytics Graph Issues#5013harshavarma29 wants to merge 1 commit intodevelopmentfrom
Conversation
…d merged role duplicates
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
HemanthNidamanuru
left a comment
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Hi Harsha,
I checked this PR locally, and everything is working fine as expected according to the requirements. However, I noticed that the date range dropdown is not aligned properly in dark mode. Could you please confirm if this issue comes under your PR?
Vishnu10-hub
left a comment
Vishnu10-hub
left a comment
There was a problem hiding this comment.
Hi,
Tested Harshavarma_Fix_Job_Posting_Analytics_Graph branch locally.
Job Analytics page loads correctly and graphs render without issues. No visual or functional regressions observed.
Feature is working as expected
Anusha-Gali
left a comment
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Harshavarma,
I have reviewed your PR locally and the functionality does work as per requirement however since this is a PR for enhancing the page, could you also please look into:
- Having some gap after the Date Range
- Having some gap after the Show % checkbox and the text
- And gap between Date Range and Show checkbox
- The Date Range dropdown is not adapted to dark mode
Note: do let me know if you would not like to make those changes in this PR, i can approve as the functionality works
4 participants
Description
Fixed the issues related to duplicate job roles, tooltip clarity, date range display in graph titles, and spacing between graphs, as reported in PR 4663
Related PRS (if any):
This frontend PR is related to the development backend PR.
Main changes explained:
How to test:
npm installandnpm run start:localto run this PR locallyVerify the following
Screenshots or videos of changes:
Job_Analytics_Video_PR.mp4